-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ARROW-6687: [Rust] [DataFusion] Add regression tests for np.nan parquet file #5502
Conversation
94f5a02
to
dac700b
Compare
@alippai Could you remove the Once the PR against the |
@andygrove changed the commit id in the |
Codecov Report
@@ Coverage Diff @@
## master #5502 +/- ##
==========================================
+ Coverage 88.66% 89.21% +0.54%
==========================================
Files 965 773 -192
Lines 128533 113051 -15482
Branches 1501 0 -1501
==========================================
- Hits 113964 100855 -13109
+ Misses 14204 12196 -2008
+ Partials 365 0 -365
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @alippai
No description provided.